[PATCH] D15668: [attrs] Split off the forced attributes utility into its own pass that can be potentially run much earlier than FuncitonAttrs proper.

David Majnemer via llvm-commits llvm-commits at lists.llvm.org
Sat Dec 19 14:04:54 PST 2015


majnemer added a subscriber: majnemer.

================
Comment at: include/llvm/Transforms/IPO/ForceFunctionAttrs.h:1
@@ +1,2 @@
+//===-- ForceFunctionAttrs.h - Remove unused function declarations -------===//
+//
----------------
Shouldn't this be 80 col long to match?

================
Comment at: lib/Transforms/IPO/ForceFunctionAttrs.cpp:15-20
@@ +14,8 @@
+#include "llvm/Transforms/IPO/ForceFunctionAttrs.h"
+#include "llvm/IR/Module.h"
+#include "llvm/IR/Function.h"
+#include "llvm/IR/LLVMContext.h"
+#include "llvm/Support/Debug.h"
+#include "llvm/ADT/StringSwitch.h"
+#include "llvm/Support/raw_ostream.h"
+using namespace llvm;
----------------
Please sort these.


http://reviews.llvm.org/D15668





More information about the llvm-commits mailing list