[PATCH] D10585: Add support for nest attribute to AArch64 backend

Tim Northover t.p.northover at gmail.com
Thu Jul 2 15:11:37 PDT 2015


> Yes, this was tested on a Release build of LLVM with assertions off.

Oh good, seems fine then.

> Are you suggesting that enabling assertions might catch a problem with this line (I manually verified the output so this shouldn't happen) or that a Release build might not emit the assembly comment?

The latter (I've been caught out by a flurry of bot complaints
before). But if you've tested it then that's fine.

Cheers.

Tim.



More information about the llvm-commits mailing list