[PATCH] D24587: Output optimization remarks in YAML

Hal Finkel via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 26 13:53:32 PDT 2016

Previous message: [PATCH] D17063: Add LLVM_FALLTHROUGH macro to Compiler.h
Next message: [PATCH] D24587: Output optimization remarks in YAML
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

hfinkel added a comment.

Great, thanks!

I'm glad you liked by suggestion for 'NV', because it is easy to type, but I do wonder if we'll have any issues because 'NV' is currently used as a variable name in a number of places. Having variable/type shadowing can certainly be confusing. What do you think?


https://reviews.llvm.org/D24587

Previous message: [PATCH] D17063: Add LLVM_FALLTHROUGH macro to Compiler.h
Next message: [PATCH] D24587: Output optimization remarks in YAML
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list