[llvm-commits] [llvm] r79484 - in /llvm/trunk: lib/Target/X86/AsmPrinter/X86ATTAsmPrinter.cpp test/CodeGen/X86/2009-08-19-inline-asm-a-modifier.ll
dalej at apple.com
Wed Aug 19 19:23:38 CDT 2009
On Aug 19, 2009, at 5:20 PMPDT, Chris Lattner wrote:
> On Aug 19, 2009, at 4:31 PM, Dale Johannesen wrote:
>> On Aug 19, 2009, at 4:06 PMPDT, Chris Lattner wrote:
>>>> URL: http://llvm.org/viewvc/llvm-project?rev=79484&view=rev
>>>> Handle 'a' modifier in X86 asms. PR 4742.
>>> hi Dale,
>>> Any reason not to make this be a FileCheck style test? If you do
>>> that, you can merge it together with other tests as well.
>> It uses a regexp. FileCheck doesn't support those, right?
> Filecheck doesn't support regex's yet, why does it need to be a regex?
Because the test has nothing to do with whatever register RA is
currently assigning, and shouldn't be dependent on it.
>>> Also, no reason to use 'else if' after a return, just use 'if'.
>> All right, I suppose, to me it reads much better this way, but it's
>> not really important.
> I'd prefer it for consistency.
More information about the llvm-commits